Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Features/fuzzy search #465

Merged
merged 7 commits into from
Oct 13, 2021
Merged

Features/fuzzy search #465

merged 7 commits into from
Oct 13, 2021

Conversation

JohanBoer
Copy link
Collaborator

Op basis van #386 hebben we deze PR gemaakt. De vorige PR was op een achterhaalde branch gedaan.
Afspraak is dat deze branch pas naar develop-1.4 (en daarna naar de master) wordt gemerged als we 100% zeker zijn dat hij in productie gaat.
Mocht deze PR niet in versie 1.4 worden geïmplementeerd dan wordt deze PR geclosed en wordt er naar develop-1.5 gemerged.

Gaarne final review

@@ -111,6 +112,7 @@ paths:
- $ref: '#/components/parameters/huisletter'
- $ref: '#/components/parameters/huisnummertoevoeging'
- $ref: '#/components/parameters/exacteMatch'
- $ref: '#/components/parameters/q'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Punt 1 onder description mag weg.
Punt 4 (wat punt 3 wordt) zou ik van maken: "Een zoekterm op postcode...."

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Description aangepast. Punt 3 vervalt niet (want nog steeds valide) en ik heb punt "4. Een zoekterm." toegevoegd.

@JohanBoer JohanBoer requested a review from strijm October 5, 2021 07:57
@JohanBoer JohanBoer merged commit 8140363 into develop-1.4 Oct 13, 2021
@strijm strijm deleted the features/fuzzy-search branch December 22, 2021 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants